Add check for non-inclusive language#64
Conversation
README.md
Outdated
| package and replaces the existing `/etc/postfix/main.cf` and | ||
| `/etc/postix/master.cf` files. Ensure to back up those files to preserve your | ||
| settings. | ||
| `/etc/postix/master.cf` files. <!--- wokeignore:rule=master --> |
There was a problem hiding this comment.
There is a typo here (was here previously) - should be /etc/postfix/master.cf - please fix that too
Add a check for usage of terms and language that is considered non-inclusive. We are using the woke tool for this with a wordlist that can be found at https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/config_files/woke.yml Signed-off-by: Noriko Hosoi <nhosoi@redhat.com>
| package and replaces the existing `/etc/postfix/main.cf` and | ||
| `/etc/postix/master.cf` files. Ensure to back up those files to preserve your | ||
| settings. | ||
| `/etc/postfix/master.cf` files. <!--- wokeignore:rule=master --> |
There was a problem hiding this comment.
@nhosoi do you know if there is a way to have an "allow" rule in woke.yml? For example, it would be good to be able to specify something like
- term: /etc/postfix/master.cf
severity: allow
- term: master
severity: error
that is, if woke found the string /etc/postfix/master.cf in any file, it would be allowed, no warnings. But other instances of the string master would be flagged.
There was a problem hiding this comment.
severity takes the value info, as well. But it's also "counted" and reported as a failure in the woke.yml action unless we apply this patch get-woke/woke#252... (;_;)
There was a problem hiding this comment.
Plus, if both master and /etc/postfix/master.cf is defined in the rule, it seems both matches.
Rule example:
rules:
- name: master.cf
terms:
- master.cf
alternatives:
- master.cf
severity: info
- name: master
terms:
- master
alternatives:
- primary
- source
- initiator,requester
- controller,host
- director
severity: warning
Woke result.
README.md:42:13-22: `master.cf` may be insensitive, use `master.cf` instead (info)
`/etc/postix/master.cf` files. Ensure to back up those files to preserve your
^
README.md:42:13-19: `master` may be insensitive, use `primary`, `source`, `initiator,requester`, `controller,host`, `director` instead (warning)
`/etc/postix/master.cf` files. Ensure to back up those files to preserve your
^
There was a problem hiding this comment.
Do we want to add another value allow to severity which completely skips the check???
There was a problem hiding this comment.
Do we want to add another value
allowtoseveritywhich completely skips the check???
Something to consider for a future version of woke, but for now, this is ok.
[1.3.2] - 2023-01-20 -------------------- ### New Features - none ### Bug Fixes - ansible-lint 6.x fixes (linux-system-roles#65) ### Other Changes - Add check for non-inclusive language (linux-system-roles#64) - cleanup non-inclusive words. Signed-off-by: Rich Megginson <rmeggins@redhat.com>
[1.3.2] - 2023-01-20 -------------------- ### New Features - none ### Bug Fixes - ansible-lint 6.x fixes (#65) ### Other Changes - Add check for non-inclusive language (#64) - cleanup non-inclusive words. Signed-off-by: Rich Megginson <rmeggins@redhat.com> Signed-off-by: Rich Megginson <rmeggins@redhat.com>
2 participants
Add a check for usage of terms and language that is considered
non-inclusive. We are using the woke tool for this with a wordlist
that can be found at
https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/config_files/woke.yml
README.md - cleanup non-inclusive words.